Fix formatting in README for 01-setup-and-first-steps#87
Fix formatting in README for 01-setup-and-first-steps#87sebastiantecsi wants to merge 3 commits intogithub:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the permissions command table in the getting-started README to correct the formatting of the /allow-all command entry.
Changes:
- Adjusted the
/allow-allcommand’s argument list formatting within the Markdown table.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| |---------|--------------| | ||
| | `/add-dir <directory>` | Add a directory to allowed list | | ||
| | `/allow-all [on|off|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status | | ||
| | `/allow-all [on\|off\|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status | |
There was a problem hiding this comment.
The backslashes in /allow-all [on\|off\|show] are inside an inline code span, so they will render literally (showing \|) rather than escaping the table delimiter. Consider reverting to /allow-all [on|off|show] inside backticks (GFM tables handle | inside code spans), or use an HTML entity for the pipe if you need to support parsers that don’t.
| | `/allow-all [on\|off\|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status | | |
| | `/allow-all [on|off|show]` | Auto-approve all permission prompts; use `on` to enable, `off` to disable, `show` to check current status | |
There was a problem hiding this comment.
In fact \| is escaping the table delimiter, and that's what we need. Otherwise /allow-all [on|off|show] renders multiple | aka table delimiters.
cc @copilot
sebastiantecsi
changed the title
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Thank you @sebastiantecsi - appreciate it! I'm working through a CodeQL issue with org admins (ruleset issue, not anything you did) that is stopping me from merging this for some reason. Once resolved I'll get it in there or cherry pick it in the worst case. Thanks. |
Uh oh!
There was an error while loading. Please reload this page.